-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
control center cdap #8
base: develop
Are you sure you want to change the base?
Conversation
cdap-e2e-tests/src/e2e-test/features/controlcenter/ControlCenter.feature
Show resolved
Hide resolved
Then Close the pipeline logs | ||
Then Click on the Hamburger bar on the left panel | ||
Then Click on Control Center link from the hamburger menu | ||
Then Verify the pipeline created by add button is succeeded successfully and is present in control center page |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change it to "Verify the pipeline created successfully is present in control center page"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to Verify the pipeline created successfully is present in control center page
cdap-e2e-tests/src/e2e-test/features/controlcenter/ControlCenter.feature
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All attributes should be control center related , Same in case of error message and pluginParam . Change if there is any common key value pair in all 4 modules which can be user as generic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok all the attributes are contol center related
@RunWith(Cucumber.class) | ||
@CucumberOptions( | ||
features = {"src/e2e-test/features"}, | ||
glue = {"io.cdap.cdap.controlcenter.stepsdesign", "stepsdesign", "io.cdap.plugin.bigquery.stepsdesign"}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is BigQuery glue added? please remove
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the glue
public static String bqSourceTable = StringUtils.EMPTY; | ||
public static String datasetName = PluginPropertyUtils.pluginProp("dataset"); | ||
|
||
@Before(order=1 ,value = "@BQ_INSERT_INT_SOURCE_TEST") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the after hook for this before hook.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added the hook
|
||
@Before(order=1 ,value = "@BQ_INSERT_INT_SOURCE_TEST") | ||
public static void createSourceBQTable() throws IOException, InterruptedException { | ||
bqSourceTable = "E2E_TARGET_" + UUID.randomUUID().toString().replaceAll("-", "_"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be source table right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CHANGED TO source
@@ -3,3 +3,7 @@ clientUrl=http://localhost:11011 | |||
serverUrl=https://placeholder.com/api | |||
# command to generate token: gcloud auth print-access-token | |||
serverAccessToken=placeholder | |||
bqSourceTable=dummy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add start and end comments for control center feature
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added the comments
6570b17
to
0df4133
Compare
0df4133
to
5c8ac3e
Compare
ControlCenter cdap-feature file